Skip to content

Abductor shuttle console fix - #5550

Merged
Rinary1 merged 1 commit into
ss14Starlight:starlight-devfrom
RedSpeeds:AbductorAccessFix
Aug 7, 2026
Merged

Abductor shuttle console fix#5550
Rinary1 merged 1 commit into
ss14Starlight:starlight-devfrom
RedSpeeds:AbductorAccessFix

Conversation

@RedSpeeds

@RedSpeeds RedSpeeds commented Aug 7, 2026

Copy link
Copy Markdown
Member

Short description

This fixes the shuttle console being locked and not usable by abductors

Why we need to add this

I was not aware that abductors had an event listener that prevent them for using any access locked systems what so ever so even if a console was not locked to anyone they couldn't use it. This PR changes the abductor shuttle so that its not locked and cannot be locked by others to fix this problem

Media (Video/Screenshots)

N/A

Checks

  • I do not require assistance to complete the PR.
  • Before posting/requesting review of a PR, I have verified that the changes work.
  • I have added screenshots/videos of the changes, or this PR does not change in-game mechanics.
  • I affirm that my changes are licensed under the MIT License and grant permission for use in this repository under its conditions.

Changelog

🆑 RedSpeeds

  • fix: Fixed Abductors not being able to interact with their shuttle console.

@RedSpeeds
RedSpeeds requested a review from a team August 7, 2026 07:59
@github-actions github-actions Bot added size/XS S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: Map Changes: No C# S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. and removed size/XS labels Aug 7, 2026
@coderabbitai

coderabbitai Bot commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 75eef23f-2645-4182-825d-2cf6a3fb9164

📥 Commits

Reviewing files that changed from the base of the PR and between 06303f3 and d0c8f5c.

📒 Files selected for processing (1)
  • Resources/Maps/_Starlight/Shuttles/ShuttleEvent/abductor_shuttle.yml
📝 Walkthrough

Summary

The Abductor shuttle console now has an unlocked Lock component. Other players cannot lock the console. Abductors can interact with the console without access-lock restrictions.

The change is limited to Resources/Maps/_Starlight/Shuttles/ShuttleEvent/abductor_shuttle.yml. The component hides lock-related verbs while keeping the console unlocked.

Gameplay impact

  • Abductors retain access to the shuttle console.
  • Other players cannot lock the console.
  • The change affects the Abductor shuttle only.

Risk areas

  • The unlocked component may affect any systems that expect the console to support locking.
  • Hidden lock verbs reduce direct player control over the console lock state.

Guideline gaps

No rationale or Starlight ownership comment is visible in the available PR context.

Walkthrough

The ComputerShuttle entity now has an unlocked Lock component. Lock-related verbs remain hidden.

Changes

Shuttle computer lock

Layer / File(s) Summary
Configure unlocked shuttle computer lock
Resources/Maps/_Starlight/Shuttles/ShuttleEvent/abductor_shuttle.yml		 Adds an unlocked Lock component to ComputerShuttle and hides lock-related verbs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: size/XS

Suggested reviewers: rinary1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: fixing the Abductor shuttle console.
Description check ✅ Passed The description explains that the console is unlocked and usable by abductors, which matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Rinary1
Rinary1 requested changes Aug 7, 2026
View reviewed changes
Comment thread Resources/Maps/_Starlight/Shuttles/ShuttleEvent/abductor_shuttle.yml
@starlightgithub starlightgithub Bot added S: Awaiting Changes Status: Changes are required before another review can happen and removed S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. labels Aug 7, 2026
Rinary1
Rinary1 approved these changes Aug 7, 2026 •
edited
Loading
View reviewed changes
@starlightgithub starlightgithub Bot added S: Approved Status: Reviewed and approved by at least one maintainer or dev; a PR may require another approval. and removed S: Awaiting Changes Status: Changes are required before another review can happen labels Aug 7, 2026
@Rinary1
Rinary1 added this pull request to the merge queue Aug 7, 2026
Merged via the queue into ss14Starlight:starlight-dev with commit e985823 Aug 7, 2026
26 of 27 checks passed
starlightgithub Bot added a commit that referenced this pull request Aug 7, 2026
@starlightgithub
Changelog: PR #5550 [skip ci]
b5d4edd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rinary1 Rinary1 Rinary1 approved these changes

Assignees

No one assigned

Labels

Changes: Map Changes: No C# S: Approved Status: Reviewed and approved by at least one maintainer or dev; a PR may require another approval. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@RedSpeeds @Rinary1